Skip to content

Include stdint.h in Utilities/Memory.h - #90

Open
jmpuch wants to merge 1 commit into
attermann:masterfrom
jmpuch:memory-stdint
Open

jmpuch wants to merge 1 commit into
attermann:masterfrom
jmpuch:memory-stdint

Conversation

@jmpuch

@jmpuch jmpuch commented Sep 26, 2026

Copy link
Copy Markdown

Problem

src/microReticulum/Utilities/Memory.h uses uint8_t, uint32_t and uint64_t without including <stdint.h>. With GCC 15 (Ubuntu 26.04, pio test -e native-local), every native build that pulls in this header fails:

src/microReticulum/Utilities/Memory.h:37:25: error: 'uint8_t' does not name a type

Older toolchains happened to get the typedefs through another header.

Change

Add #include <stdint.h> to Memory.h. No functional change.

Testing

  • Native (Linux, GCC 15.2): pio test -e native-local for test_crypto, test_transport, test_rns_loopback, test_bytes, test_general now build and pass.
  • ESP32 (ttgo-lora32-v21) and nRF52 builds are unaffected (the include is a no-op where the types were already visible).

🤖 Generated with Claude Code

Memory.h uses uint8_t/uint32_t/uint64_t without including <stdint.h>;
native builds with GCC 15 fail with "'uint8_t' does not name a type".

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant